Skip to content

Tickets/sp 2237#25

Closed
DouglasLeeTucker wants to merge 8 commits into
mainfrom
tickets/SP-2237
Closed

Tickets/sp 2237#25
DouglasLeeTucker wants to merge 8 commits into
mainfrom
tickets/SP-2237

Conversation

@DouglasLeeTucker
Copy link
Copy Markdown
Contributor

Let's do this! :-)

@review-notebook-app
Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Copy Markdown

@gpdf gpdf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small changes to clarify the text to match the changes made to the code. Please re-request review once you've pushed your changes.

Comment thread DP0.2/02c_Image_Queries_with_TAP.ipynb Outdated
Comment thread DP0.2/02c_Image_Queries_with_TAP.ipynb Outdated
Comment thread DP0.2/02c_Image_Queries_with_TAP.ipynb Outdated
Comment thread DP0.2/02c_Image_Queries_with_TAP.ipynb Outdated
Comment thread DP0.2/02c_Image_Queries_with_TAP.ipynb Outdated
Copy link
Copy Markdown

@gpdf gpdf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@DouglasLeeTucker DouglasLeeTucker deleted the tickets/SP-2237 branch June 25, 2025 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants